Skip to content

Adding Clock lambda helper#279

Closed
severn-everett wants to merge 2 commits intoKotlin:masterfrom
severn-everett:add-clock-function
Closed

Adding Clock lambda helper#279
severn-everett wants to merge 2 commits intoKotlin:masterfrom
severn-everett:add-clock-function

Conversation

@severn-everett
Copy link
Copy Markdown

@severn-everett severn-everett commented Jun 6, 2023

Added helper function to create a Clock instance by passing in a lambda to be invoked for Clock.now(); Updated Kotlin shield in README to reflect usage of version 1.8.10

Fixes #273

…da to be invoked for Clock.now(); Updated Kotlin shield in README to reflect usage of version 1.8.10
@dalewking
Copy link
Copy Markdown

I prefer the idea of just making Clock be a "fun" interface which accomplishes the same thing.

@dkhalanskyjb
Copy link
Copy Markdown
Collaborator

kotlinx-datetime removes Clock in favor of kotlin.time.Clock, so this is no longer applicable. In any case, in my opinion, the presented use case is better covered by https://youtrack.jetbrains.com/issue/KT-85461/Add-kotlin.time.TestClock

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clock interface should be a "fun" interface

3 participants